wayland: stage uncommitted changes to dedicated buffer

Right now we use one buffer for both staged changes (freshly painted
changes waiting for the frame clock to send to the compositor) and
committed changes (changes actively being read by the compositor
process). This creates a problem in the event we need to stage updates
at the same time the compositor is processing committed updates: we
can't change what the compositor is actively processing.

The current solution for handling this contention is to allocate a
temporary buffer on the spot at the time the updates are staged, and to
copy that buffer back to the shared buffer later.  The problem, though,
is that the copy to the shared buffer currently happens as soon as
the updates are finished being staged, not when the shared buffer is
done being processed by the compositor.

In order to address that problem, this commit changes the code to always
stage changes to a dedicated staging buffer.  The staging buffer is
used exclusively by the client until the client is done with it, and then
once that staging buffer is committed, the client never writes to that
buffer again.  If the client needs to stage new updates, it allocates a
brand new staging buffer, draws to it, and back fills the undrawn parts
of the buffer from a copy of the contents of the committed buffer.

As an optimization, the compositor has the option of releasing the
committed buffer back to the client.  If it does so before the client
needs to stage new updates, then the client will reuse the buffer
for staging future updates.  This optimization prevents having to allocate
a new staging buffer and the associated cost of back filling
that new buffer with a readback of the committed buffer.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: don't handle buffer release centrally

Right now we handle buffer releases coming from the
compositor in a central place. We add a listener when
first creating the shared buffers.

This is problematic because a buffer can only have
one listener on it at once so users of the buffer
can't get notified when it's released.

This commit moves the buffer listener code from the
centrally managed display code to the cursor and window
code.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: always return FALSE from begin_paint

The client and compositor share access to the window
pixel buffers. After the client hands off (commits)
the buffer to the compositor it's not supposed to write
to it again until it's released by the compositor.

The code tries to deal with this contention by allocating
a temporary buffer and using that in the mean time. This
temporary buffer is allocated by a higher layer of the code
when begin_paint returns TRUE. Unfortunately, that layer of
the code has no idea when the buffer is released, so it ends
up blitting the temporary buffer back to the shared buffer
prematurely.

This commit changes begin_paint to always return FALSE.

A future commit will address the contention problem in
a different way.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: use g_clear_pointer when destroying cairo surfaces

There are a few places where we destroy a cairo surface and
then nullify it. This commit changes those to use
g_clear_pointer instead.

It also drops a cairo_surface_finish call that is unnecessary

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: rename cairo surface user data key to be more specific

This commit renames the key name to be more specific for clarity.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: move server proxy objects to substructure

This commit moves the server proxy objects to a substructure
for clarity.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

wayland: rename window->surface to window->wl_surface

The name surface is really overloaded when dealing
with wayland windows.

To alleviate ambiguity, this commit changes the name
of the "surface" and "subsurface" members to have
a wl_ prefix.

https://bugzilla.gnome.org/show_bug.cgi?id=761312

Fix stylecontext tests

This was broken by f7ec9c98ef0ef8740c93f96af9d971b0211118c1,
since type names are no longer used at all in CSS matching.

Update CSS docs regarding type names

We no longer use type names at all.

cssselector: Remove type references

We don't use types anymore.

cssmatcher: Remove type argument from name matcher

We no longer use types when matching names.

inspector: Drop type name from node list

It's not used anymore in the CSS, so people don't need to see it.

cssimageradial: Only allow at "<position>" after other props

Don't allow syntax like
  at top left circle
but follow the spec about requiring the at <position> right before the
comma.

This is porbably because
  circle at 10px 10px
could be interpreted as
  circle 10px at 10px
with the now disallowed syntax, too.

Test included.

cssimageradial: Only emit one error for wrong position

Test included.

css tests: Fix style tests

Commit d0fd7990c3f2c738f86185418ccd76b14da8a776 changed
the location of builtin themes, breaking the loading of
the empty theme in test-css-style.

css tests: Fix expected output

This is fallout from f7ec9c98ef0ef8740c93f96af9d971b0211118c1.

css: Convert numbers with GTK's dpi

Don't hardcode 96 for dpi, but instead use the value of the -gtk-dpi
property (that mirrors the GdkScreen's dpi if it wasn't set explicitly).

This makes these values scale when the large font setting in
control-center is enabled.

css: Properly convert em, ex and rem to pixels

The previous code converted to pt.

Adwaita: darker background for popovers

levelbar: Make discrete levelbars behave as before

During the gadget conversion, the drawing of discrete levelbars
was unintentionally changed to draw a wide trough but narrow
blocks, which does not look great. So go back to the previous
way of drawing things.

https://bugzilla.gnome.org/show_bug.cgi?id=761428

Fix indentation

gcc 6 complains about this.

Document rem as CSS unit

switch: Don't queue draws after setting state flags

css: Add rem unit

Good luck, Lapo.

widget: Set a css name

Now selecting a widget by class name no longer works.

This is probably most relevant for users outside of GTK that want to
style their own widgets. Those widgets should now either add their own
style classes (if they want to adjust existing CSS) or use
gtk_widget_class_set_css_name() themselves (if they want to get rid of
all "upstream" styling).

Range: Group all three gestures

And reset the grab_location in the ::released handler of the multipress
gesture.

Previously, when leaving fine-tune mode, the ::released handler of the
multipress gesture would call stop_scrolling, which calls
range_grab_remove and resets the grab_location. The ::drag-end handler
is executed after that, and only unsets priv->in_drag if the
grab_location is MOUSE_OUTSIDE, which it never was, since the ::released
handler already reset it. This lead to priv->in_drag being set even
though no dragging was in progress anymore, which e.g. made shift
pressed after leaving the fine-tune mode entering it again.

https://bugzilla.gnome.org/show_bug.cgi?id=761402

x11: don't apply scaling factor twice

The fallback behaviour of get_work_area () divides the
screen width and height by the window scaling factor, but
those values are already scaled down.

https://bugzilla.gnome.org/show_bug.cgi?id=761474

Don't do remote checks on NULL files

Calling _gtk_file_consider_as_remote() with a NULL argument
results in warnings being thrown.

Note that query->priv->location being NULL is a state that does
not seem to be invalid by itself.

This could happen if you do search-as-you-type in a filechooser,
which has a filter that does not match anything *and* the current
"place" selected is "Recent".

https://bugzilla.gnome.org/show_bug.cgi?id=761552

Move private widget path functions away

We don't put private functions in public headers anymore.

https://bugzilla.gnome.org/show_bug.cgi?id=761339

win32: Get rid of XP theme

XP is not supported anymore. And we can massively simplify the win32
theme if we don't have to support 2 versions.

win32: Delete classic theme

Well, actually, stop adding an extra version of raleigh

build: Don't break windows build

Hardcode the existing files instead of doing a for loop and attempting
string-fudging magic.

Adwaita: removed some old cruft from _drawing.scss

resulting in a nice number of non working selectors ditched.

inspector: Don't hardcode builtin themes

Insteead, enumerate the resources that are available. Fixes windows
themes not being listed in the inspector.

cssprovider: Change location of builtin themes

Instead of
  /org/gtk/libgtk/theme/$THEME-$VARIANT.css
look at
  /org/gtk/libgtk/theme/$THEME/gtk-$VARIANT.css
and that way mirror the directory layout of real themes.

actionbar: Remove unnecessary map/unmap implementations

These just do what the default implementations do anyway.

statusbar: Remove unused include

Remove _gtk_box_get_children

It's doing exactly the same thing as gtk_container_get_children.

colorswatch: Remove unused constant

entry: queue an allocate when pulsing

We need to reposition the progress gadget and that's done during
allocation.

gdkwindow-x11: always set _GTK_THEME_VARIANT property

Set _GTK_THEME_VARIANT to empty string when default theme variant
is used. This will allow to understand whether _GTK_THEME_VARIANT
is not supported or default variant is requested.

https://bugzilla.gnome.org/show_bug.cgi?id=761476

gtksettings: fix critical warning

Commit 5186aeb90f52c941a2642b04ebfb54e9d8f8ea11 introduced critical
warning - g_value_copy: assertion 'g_value_type_compatible (...)' failed.

https://bugzilla.gnome.org/show_bug.cgi?id=761381

broadway: Define variables where they are used

Avoids gcc warnings about unused variables.

composetable: Fix gcc warnings

immodule: Use correct sign for function prototype

entry: Turn progress node into a gadget

entry: Simplify drawing code

Adwaita: fix headerbar spinbutton size

fixes https://bugzilla.gnome.org/show_bug.cgi?id=761484

gdkborder: remove unneeded definition

This reverts 54fa65f544bef281640d18a4a725005f6199a956,
5190d010d95010a845715c719c3b85a75e0c513a, and
a6e4de28842002e7bfdf73def26c4c52d9f9126a.

https://bugzilla.gnome.org/show_bug.cgi?id=756579

print dialog: Fix layout preview drawing

Use the CSS name paper for this as well, and make sure Adwaita
sets a color that makes the text visible in the dark theme as well.

Adwaita: notebook with hidden header style fix

the stack styling is applied only when there is a header, hence
when the stack node is not the notebook only child, should cure
https://bugzilla.gnome.org/show_bug.cgi?id=760996

Adwaita: fix vertical spinbutton style

correct sizing and workaround disabled entry text colors.

notebook: Do not reorder tabs after a grab notify if there isn't an ongoing reorder operation

This broke in f535251507a6a0af68aed6348ae80cbdac9b1f4a that removed
during_reorder in favor of using the current operation, but removed the
check from gtk_notebook_stop_reorder().

Updated French translation
(cherry picked from commit 85fb4c1c0a17906a65e0b732770b88353b6214c4)

gtksettings: reset property to correct default value

https://bugzilla.gnome.org/show_bug.cgi?id=76138

Revert "Add missing (out) annotation to gdk_rgba_parse"

This reverts commit 695860958a9a2e7c825bcec4d0d42b427f0fb47e.

The change introduced an introspection ABI break.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=682125

If we want to move the parse() method to be a constructor or a class
method, we'll need to do more testing or add new API.

https://bugzilla.gnome.org/show_bug.cgi?id=754990

entry: Draw progress in the right place

Updated Hungarian translation

Updated German translation

reftests: Make separator-size test pass in HighContrastInverse

After neutralizing the stack background, the notebook itself
was peeking through.

Updated Spanish translation

Updated Spanish translation

Trivial formatting fix

reftests: Make label-text-shadow-changes-modify-clip test more reliable

This test was failing in continuous, where the tests are running
in a VM with disabled animations. Make the test adapt to that
situation by changing the rui on the fly if we find that animations
are disabled.

reftests: Add a message to the test output

Emit a message that ends up in the test output if we are modifying
the test on the fly.

reftests: Rename a source file

Easier to keep track of things if the sources are names after
the test they are used in.

entry: Fix icon areas

Do the same thing we already do in gtk_entry_get_text_area.

widget: Annotate out params of _translate_coordinates as optional

reftests: Make animation-direction test more reliable

This test was failing in continuous, where the tests are running
in a VM with disabled animations. Make the test adapt to that
situation by changing the reference ui on the fly if we find
that animations are disabled.

gtk-builder-tool: Expand the preview command

Make the preview command parse options properly, turn the ID into
an --id=ID option, and add a --css=FILE option that allows to
specify a css file to use for previewing.

This is useful for e.g. previewing the reftest .ui files with
their corresponding .css.

gtk-launch: Add --version

Support --version. Also improve the --help output cosmetically.

gtk-builder-tool: Accept --help

Saying 'Failed to open file '--help' is not polite or smart.
Catch this case and show usage information.

file chooser: Avoid a crash

Since 39c2d12330b6d4405ca8a5599c12017c58626fcf,
priv->operation_mode == OPERATION_MODE_BROWSE no longer
guarantees that priv->browse_files_model is the current
model of the list - we are only switching the models after
loading the new directory. Avoid triggering the assertion
in show_and_select_files by checking if we have right model
before calling it.

https://bugzilla.gnome.org/show_bug.cgi?id=761209

Don't include the builtin sequence table in multiple places

The table is defined as static const guint16 [], so it really
should only be included in one source file.

combobox: Avoid a misleading indentation

gcc 6 warns about this sort of thing. There were also some
stray ; here.

Fix the border-image-excess-size reftest

This was failing because the grid is now respecting border
width too. Making the selector more specific so it only
applies to the toolbars fixes this.

Fix another reftest

Make sure the notebook in the separator-size reftest does not
peek through, by giving it a transparent background.

Fix some reftests under wayland

Tests need to use popup windows, in order to avoid differences
due to CSD.

wayland: Reduce warning level for parentless temporaries

Temp windows without parent are used frequently in our testsuite;
using g_warning on them causes the tests to fail, which is not
useful. Reduce the warning to a g_message.

treeview: Draw background when empty

We have a function we call for drawing the empty case, but
it only renders a focus rectangle. Make it draw the background
as well.

https://bugzilla.gnome.org/show_bug.cgi?id=761309

colorchooserwidgeet: Reset the selected swatch when removing

Otherwise, priv->current points to garbage and subsequently leads to a
crash, e.g. when adding another custom color.

Document gtk_parse_args limitations

Mention that some things won't work without a display
connection.

Throw an error when a display is missing

Creating style context won't work without a display, so
give a clear error message and abort instead of segfaulting
later on.

https://bugzilla.gnome.org/show_bug.cgi?id=761332

3.19.8

Remove entry-progress-coloring reftest

We don't recolor text over progress anymore, so this
tests a no longer existing feature.

Use TAP output for installed tests

Run reftests with HighContrastInverse too

For installed tests, we run the reftests with all the builtin
themes (minus platform-specific ones). Add HighContrastInverse
to that list.

Adwaita: Add symbolic png assets

And use them as fallback for the svg ones. This lets us
avoid a hard dependency on librsvg - the theme will work
without svg support.

icontheme: Avoid a crash

gtk_icon_info_new_for_file with a size of 0 would fail for
symbolic pngs because we ended up scaling with 0. Avoid this.

css: Support the image() notation

This lets us do fallback in case an image format is not
supported, and also lets us provide solid-color images.
We don't support image fragment notations.

See ttps://www.w3.org/TR/css3-images/#image-notation

https://bugzilla.gnome.org/show_bug.cgi?id=761318

Emit an error if image loading fails

Instead of spamming stderr with g_warning, use the new
emit_error method of the GtkStyleProviderPrivate interface
to emit an error if loading an image fails.

CSS provider: Add a way to emit errors

Currently, GtkCssProvider can emit ::parsing-error only during
the actual parsing, although the documentation hints that it might
happen at other times.

This commit adds a emit_error method to the GtkStyleProviderPrivate
interface that will let us emit errors from the compute() implementations
as well, which can be useful (e.g. if an image fails to load).

Updated Hungarian translation

Updated Hungarian translation

Updated Hungarian translation

file chooser: Make Escape close the dialog

In some situations (no header bar, save mode), hitting Escape
would not do anything because the entry ate the key event.
Fix this by telling the entry to only handle Escape when there
is something to do, such as switching back to the path bar.

https://bugzilla.gnome.org/show_bug.cgi?id=761026

Add a css style test for color names

Verify that we support all the css color names, with their expected
rgb values.

css image recolor: Load images at the proper scale

We should not hardcode a scale of 1, this leads to
pixellated upscaled images at scale=2, even if the source
is an svg. By passing the proper scale, we can load the
svg at the correct size.

icon theme: Improve loading of unthemed files

When creating icon info objects for unthemed files, we don't
really have a nominal size, so we pass 0 to mean 'load at
original size'. However, this is not what was happening.
To make this possible, add variants of some pixbuf loading
functions that take a scale factor instead of a desired size,
and use those when we don't have a nominal size.

image: use GtkMisc padding again

This was lost in the gadget conversion.
Thankfully, the misc-alignment reftest catches this.